-
Notifications
You must be signed in to change notification settings - Fork 24.7k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Split compiler and core #18683
Split compiler and core #18683
Conversation
You can preview 1e57ea0 at https://pr18683-1e57ea0.ngbuilds.io/. |
1e57ea0
to
0aefd15
Compare
0aefd15
to
f422a63
Compare
You can preview 0aefd15 at https://pr18683-0aefd15.ngbuilds.io/. |
You can preview f422a63 at https://pr18683-f422a63.ngbuilds.io/. |
this._registerDecoratorOrConstructor( | ||
this.findDeclaration(ANGULAR_CORE, 'Component'), Component); | ||
this._registerDecoratorOrConstructor(this.findDeclaration(ANGULAR_CORE, 'NgModule'), NgModule); | ||
this._registerDecoratorOrConstructor(MetadataIdentifier.Host, createHost); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
As discussed offline, I don't believe MetadataIdentifer
is required.
@@ -0,0 +1,149 @@ | |||
/** |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Consider moving the reflect_metadata
requirement to this module so we no longer require it in compiler
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would like to postpone this to another PR...
f422a63
to
c1e5781
Compare
You can preview c1e5781 at https://pr18683-c1e5781.ngbuilds.io/. |
c1e5781
to
cbdafd2
Compare
cbdafd2
to
20d14e2
Compare
You can preview 20d14e2 at https://pr18683-20d14e2.ngbuilds.io/. |
20d14e2
to
b4bc466
Compare
You can preview b4bc466 at https://pr18683-b4bc466.ngbuilds.io/. |
After this, neither @angular/compiler nor @angular/comnpiler-cli depend on @angular/core. This add a duplication of some interfaces and enums which is stored in @angular/compiler/src/core.ts BREAKING CHANGE: - `@angular/platform-server` now additionally depends on `@angular/platform-browser-dynamic` as a peer dependency.
b4bc466
to
40754da
Compare
Note: This was already green, CI only runs because of another rebase. |
You can preview 40754da at https://pr18683-40754da.ngbuilds.io/. |
This issue has been automatically locked due to inactivity. Read more about our automatic conversation locking policy. This action has been performed automatically by a bot. |
Note: The first 4 commits are from #18720